-
Notifications
You must be signed in to change notification settings - Fork 0
Conversation
This new menu section will display measurements of points and distances, and display extra attributes
… for measurements to the same tab Closes #5
Closes #5
Added title for the measurement list, and for the data display. Moved some options from scene to measurements tab. Added opening and closing transition to the tab
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, the UI looks really good. Made some comments. Also, which script did you use inn order to keep attributes (e.g. acceptance, TVU, THU) during the conversion?
|
@adriahso I have the converter script outside of the repo. Its basically the script that Tobias made a while ago, with some adjustments. Do you need it? |
Well, without it I can't really see the attributes of our data like accepted, TVU and THU, which would be nice. Maybe just put the script somewhere in the Google Drive? |
|
Uploaded to google drive now: converter.py |
|
Nice, could see it now. Some minor things:
|
Yeah the measurements data title was useless, so I removed it. Should look cleaner now. Also cleaned up the UI (titles/dividers and spacing) Let me know if there is something else I've missed. This files turned out larger than I expected, which is due to making a custom UI for the measurements and also because volumes and profiles are handled differently, but I'm sure the code can be cleaner. If time allows it i want to revisit this at the end of the project (which it probably wont) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks very good to me!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
No description provided.